Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The virtual keyboard should show or hide automatically if enabled #1086

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kingysu
Copy link
Contributor

@kingysu kingysu commented Jul 6, 2024

  1. When the user interacts with some applications rather than the virtual keyboard itself, the virtual keyboard should show or hide if it is enabled.

  2. For example, if the user clicks a window and the window has a text input control which has the focus and accepts the input from the keyboard, then the virtual keyboard should show automatically if the auto show switch is enabled.

  3. Then, if the window loses the focus, the virtual keyboard should hide automatically if the auto hide switch is enabled.

@wengxt
Copy link
Member

wengxt commented Jul 12, 2024

I'm not fully understand the motivation of this change here.

To me, it seems like the "is auto hide" mode check is pushed down to the UI level. Which seems to be wrong to me.

UI's API should be simply pure, when show/hide is called, then it just show or hide, without checking any other "auto" condition.

As your issues, I think the issue should be solved in other some way.

1. When the user interacts with some applications rather than the
virtual keyboard itself, the virtual keyboard should show or hide
if it is enabled.

2. For example, if the user clicks a window and the window has a
text input control which has the focus and accepts the input from
the keyboard, then the virtual keyboard should show automatically
if the auto show switch is enabled.

3. Then, if the window loses the focus, the virtual keyboard should
hide automatically if the auto hide switch is enabled.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants